Skip to content

Conversation

@AdityaHegde
Copy link
Collaborator

@AdityaHegde AdityaHegde commented Jan 7, 2026

Adds a tooltip to leaderboards similar to dimension table that states that a value can be copied by pressing shift+click. Hides the tooltip after 3s to avoid blocking the rest of the data.

Closes APP-657

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@ericokuma
Copy link
Contributor

ready for review @AdityaHegde?

@AdityaHegde
Copy link
Collaborator Author

@ericokuma Yes, this is ready for a UXQA

@ericokuma
Copy link
Contributor

Looks good! Would be great if we can add the same copy to the Measures as well:

Screenshot 2026-01-14 at 2 13 23 PM

since copy + click works there as well

@AdityaHegde AdityaHegde force-pushed the feat/value-copy-tooltip-in-leaderboards branch from 0c77adb to 250fcde Compare January 15, 2026 07:17
@AdityaHegde AdityaHegde marked this pull request as ready for review January 15, 2026 07:18
@AdityaHegde
Copy link
Collaborator Author

@ericokuma added to big measure as well

Copy link
Member

@djbarnwal djbarnwal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Unrelated to the changes in this PR but there seems to be a regression in how copy works for big number values. When hovering over a comparison delta, I see the delta in the tooltip but when copying, the current period value is copied.

image

@AdityaHegde
Copy link
Collaborator Author

Thanks @djbarnwal lets fix that as a separate PR

@AdityaHegde AdityaHegde merged commit ef75fcd into main Jan 16, 2026
11 checks passed
@AdityaHegde AdityaHegde deleted the feat/value-copy-tooltip-in-leaderboards branch January 16, 2026 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants